ci(warden): Refine PR review checks#438
Merged
Merged
Conversation
Narrow PR-time Warden coverage to a smaller fast lane with explicit budgets. Move broad or noisy skills out of normal PR cadence and pin remote Warden skills/actions so check behavior is reproducible. Add workflow timeout and concurrency guardrails so obsolete Warden runs do not continue delaying PR feedback after newer commits arrive. Co-Authored-By: OpenAI Codex <noreply@openai.com>
commit: |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix prepared a fix for the issue found in the latest run.
- ✅ Fixed: Find-bugs skips runtime entrypoints
- Added missing runtime entrypoint files (cli.ts, daemon.ts, doctor-cli.ts) and src/mcp/resources/** directory to the find-bugs paths array.
Or push these changes by commenting:
@cursor push 421b9e90cf
Preview (421b9e90cf)
diff --git a/warden.toml b/warden.toml
--- a/warden.toml
+++ b/warden.toml
@@ -314,9 +314,13 @@
maxTurns = 10
maxFindings = 5
paths = [
+ "src/cli.ts",
"src/cli/**",
+ "src/daemon.ts",
"src/daemon/**",
+ "src/doctor-cli.ts",
"src/integrations/**",
+ "src/mcp/resources/**",
"src/mcp/tools/**",
"src/runtime/**",
"src/server/**",You can send follow-ups to the cloud agent here.
Reviewed by Cursor Bugbot for commit a0e59cf. Configure here.
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
Include runtime entrypoints and MCP resources in the remaining PR fast-lane Warden checks so source-only changes to those surfaces still get reviewed. Let Warden sweep runs queue instead of cancelling in-progress repository mutation work. Co-Authored-By: OpenAI Codex <noreply@openai.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Refine Warden's PR-time review setup so routine PRs get faster, higher-signal feedback.
This keeps Warden in the PR loop, but narrows it to an explicit fast lane: project-specific checks, one bounded generic bug finder, and targeted security/workflow checks. Broad or low-signal reviews move out of normal PR cadence, and every remaining PR skill gets explicit budgets so neutral/no-op checks do not consume unbounded feedback time.
The PR also adds workflow timeout/concurrency guardrails and pins Warden-related actions/remotes to immutable refs. This should make Warden behavior more reproducible and prevent obsolete runs from continuing after newer commits arrive.
Notes for review:
wrdn-piiis local-only for now because recent PR history showed a Pi model selector failure.find-bugsstays on PRs, but is narrowed to higher-risk runtime/tooling paths.code-reviewmoves to scheduled review, andcode-simplifierbecomes local-only cleanup.